Skip to content

Remove unused dependency region tags and update used region tags #1197

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Aug 29, 2018

Conversation

alixhami
Copy link
Contributor

The goal with sample region tags is to have a globally unique tag for each sample. The "dependencies" tag is used throughout the java-docs-samples repo, which goes against best practices. This PR accomplishes the following:

  • For region tags that are not used on cloud.google.com, remove the region tag
  • For region tags that are used on cloud.google.com, add a prefix of product_java_ (ex. bigquery_java_dependencies)

Note that this PR does not include all products, only ones that I am responsible for adding to the sample tracker. Others will be performing this update for the remaining products.

@alixhami alixhami requested review from tswast and nnegrey August 27, 2018 21:38
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Aug 27, 2018
nnegrey
nnegrey previously approved these changes Aug 27, 2018
Copy link
Contributor

@nnegrey nnegrey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
(interesting how some are used and others aren't)

@nnegrey nnegrey added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Aug 27, 2018
tswast
tswast previously approved these changes Aug 28, 2018
@alixhami alixhami dismissed stale reviews from tswast and nnegrey via 97329cd August 28, 2018 21:28
@alixhami alixhami force-pushed the update-dependency-tags branch from 5e63c84 to 97329cd Compare August 28, 2018 21:28
@alixhami
Copy link
Contributor Author

I updated my approach with the unused region tags. Rather than deleting them, I gave them names that use the standard globally unique naming conventions, so that we avoid people adding the plain "dependencies" region tag in the future.

@alixhami alixhami force-pushed the update-dependency-tags branch from 97329cd to 4669400 Compare August 29, 2018 00:38
tswast
tswast previously approved these changes Aug 29, 2018
@nnegrey nnegrey added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 29, 2018
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 29, 2018
@alixhami alixhami changed the title [Do not merge] Remove unused dependency region tags and update used region tags Remove unused dependency region tags and update used region tags Aug 29, 2018
@alixhami alixhami removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Aug 29, 2018
@alixhami alixhami force-pushed the update-dependency-tags branch from 4669400 to 2f10893 Compare August 29, 2018 19:32
@tswast tswast merged commit a88e2ff into master Aug 29, 2018
@alixhami alixhami deleted the update-dependency-tags branch August 29, 2018 20:00
Shabirmean pushed a commit that referenced this pull request Nov 17, 2022
…1197)

[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [com.google.cloud:google-cloud-core](https://togithub.com/googleapis/java-core) | `2.8.5` -> `2.8.6` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.8.6/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.8.6/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.8.6/compatibility-slim/2.8.5)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.8.6/confidence-slim/2.8.5)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>googleapis/java-core</summary>

### [`v2.8.6`](https://togithub.com/googleapis/java-core/blob/HEAD/CHANGELOG.md#&#8203;286-httpsgithubcomgoogleapisjava-corecomparev285v286-2022-08-02)

[Compare Source](https://togithub.com/googleapis/java-core/compare/v2.8.5...v2.8.6)

##### Dependencies

-   update dependency com.google.api:gax-bom to v2.18.7 ([#&#8203;890](https://togithub.com/googleapis/java-core/issues/890)) ([c67bed5](https://togithub.com/googleapis/java-core/commit/c67bed5281b458e78780cdd918235b5d073917c7))
-   update dependency com.google.code.gson:gson to v2.9.1 ([#&#8203;888](https://togithub.com/googleapis/java-core/issues/888)) ([7b799c6](https://togithub.com/googleapis/java-core/commit/7b799c66f4aaf057242e9a231b920e49a7c964e9))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox.

---

This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-automl).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzMi4xMzUuMSIsInVwZGF0ZWRJblZlciI6IjMyLjEzNS4xIn0=-->
Shabirmean pushed a commit that referenced this pull request Nov 18, 2022
…1197)

[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [com.google.cloud:google-cloud-core](https://togithub.com/googleapis/java-core) | `2.8.5` -> `2.8.6` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.8.6/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.8.6/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.8.6/compatibility-slim/2.8.5)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.8.6/confidence-slim/2.8.5)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>googleapis/java-core</summary>

### [`v2.8.6`](https://togithub.com/googleapis/java-core/blob/HEAD/CHANGELOG.md#&#8203;286-httpsgithubcomgoogleapisjava-corecomparev285v286-2022-08-02)

[Compare Source](https://togithub.com/googleapis/java-core/compare/v2.8.5...v2.8.6)

##### Dependencies

-   update dependency com.google.api:gax-bom to v2.18.7 ([#&#8203;890](https://togithub.com/googleapis/java-core/issues/890)) ([c67bed5](https://togithub.com/googleapis/java-core/commit/c67bed5281b458e78780cdd918235b5d073917c7))
-   update dependency com.google.code.gson:gson to v2.9.1 ([#&#8203;888](https://togithub.com/googleapis/java-core/issues/888)) ([7b799c6](https://togithub.com/googleapis/java-core/commit/7b799c66f4aaf057242e9a231b920e49a7c964e9))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox.

---

This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-automl).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzMi4xMzUuMSIsInVwZGF0ZWRJblZlciI6IjMyLjEzNS4xIn0=-->
anguillanneuf pushed a commit that referenced this pull request Dec 5, 2022
…1197)

[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [com.google.cloud:google-cloud-core](https://togithub.com/googleapis/java-core) | `2.8.5` -> `2.8.6` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.8.6/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.8.6/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.8.6/compatibility-slim/2.8.5)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.8.6/confidence-slim/2.8.5)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>googleapis/java-core</summary>

### [`v2.8.6`](https://togithub.com/googleapis/java-core/blob/HEAD/CHANGELOG.md#&#8203;286-httpsgithubcomgoogleapisjava-corecomparev285v286-2022-08-02)

[Compare Source](https://togithub.com/googleapis/java-core/compare/v2.8.5...v2.8.6)

##### Dependencies

-   update dependency com.google.api:gax-bom to v2.18.7 ([#&#8203;890](https://togithub.com/googleapis/java-core/issues/890)) ([c67bed5](https://togithub.com/googleapis/java-core/commit/c67bed5281b458e78780cdd918235b5d073917c7))
-   update dependency com.google.code.gson:gson to v2.9.1 ([#&#8203;888](https://togithub.com/googleapis/java-core/issues/888)) ([7b799c6](https://togithub.com/googleapis/java-core/commit/7b799c66f4aaf057242e9a231b920e49a7c964e9))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox.

---

This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-automl).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzMi4xMzUuMSIsInVwZGF0ZWRJblZlciI6IjMyLjEzNS4xIn0=-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants